Skip to content

Conversation

@felixfontein
Copy link
Collaborator

SUMMARY

While starting to add more type hints I noticed that some module utils aren't used at all. Let's deprecate them so we can remove them soon.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

plugins/module_utils/cloud.py
plugins/module_utils/database.py
plugins/module_utils/known_hosts.py
plugins/module_utils/saslprep.py

@felixfontein felixfontein added check-before-release PR will be looked at again shortly before release and merged if possible. backport-12 Automatically create a backport for the stable-12 branch labels Nov 24, 2025
@ansibullbot ansibullbot added feature This issue/PR relates to a feature request module_utils module_utils plugins plugin (any type) labels Nov 24, 2025
@felixfontein felixfontein requested a review from russoz November 24, 2025 19:50
@ansibullbot ansibullbot added the needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR label Nov 24, 2025
Copy link
Collaborator

@russoz russoz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@felixfontein felixfontein removed the check-before-release PR will be looked at again shortly before release and merged if possible. label Nov 25, 2025
@felixfontein felixfontein merged commit d364e35 into ansible-collections:main Nov 25, 2025
132 of 134 checks passed
@patchback
Copy link

patchback bot commented Nov 25, 2025

Backport to stable-12: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-12/d364e35423b6a105753654747cce2ad9b75704db/pr-11205

Backported as #11213

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Nov 25, 2025
Deprecate unused module utils.

(cherry picked from commit d364e35)
@felixfontein
Copy link
Collaborator Author

@russoz thanks for reviewing!

@felixfontein felixfontein deleted the cleanup branch November 25, 2025 05:50
felixfontein added a commit that referenced this pull request Nov 25, 2025
#11213)

Deprecate unused module utils (#11205)

Deprecate unused module utils.

(cherry picked from commit d364e35)

Co-authored-by: Felix Fontein <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-12 Automatically create a backport for the stable-12 branch feature This issue/PR relates to a feature request module_utils module_utils needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR plugins plugin (any type)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants